Skip to content

chore: remove deps using machete#7146

Closed
joseph-isaacs wants to merge 1 commit intodevelopfrom
ji/remove-dep
Closed

chore: remove deps using machete#7146
joseph-isaacs wants to merge 1 commit intodevelopfrom
ji/remove-dep

Conversation

@joseph-isaacs
Copy link
Copy Markdown
Contributor

Remove some deps

Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
@joseph-isaacs joseph-isaacs marked this pull request as draft March 24, 2026 16:53
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 24, 2026

Merging this PR will degrade performance by 24.79%

⚡ 7 improved benchmarks
❌ 7 regressed benchmarks
✅ 1087 untouched benchmarks
⏩ 1522 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation new_bp_prim_test_between[i64, 32768] 178.6 µs 237.5 µs -24.79%
Simulation old_alp_prim_test_between[f32, 32768] 402.3 µs 447.4 µs -10.08%
Simulation patched_take_10k_contiguous_not_patches 279.7 µs 249.1 µs +12.28%
Simulation new_bp_prim_test_between[i64, 16384] 114.7 µs 144.2 µs -20.46%
Simulation patched_take_10k_contiguous_patches 279.4 µs 248.9 µs +12.25%
Simulation patched_take_10k_dispersed 328 µs 297.5 µs +10.24%
Simulation new_alp_prim_test_between[f64, 16384] 126.9 µs 148.9 µs -14.78%
Simulation patched_take_10k_first_chunk_only 313.8 µs 283.3 µs +10.79%
Simulation take_10k_first_chunk_only 274.2 µs 229.1 µs +19.67%
Simulation take_10k_dispersed 288.1 µs 243.1 µs +18.53%
Simulation patched_take_10k_random 291 µs 260.5 µs +11.73%
Simulation new_bp_prim_test_between[i32, 16384] 97.9 µs 112.4 µs -12.89%
Simulation new_bp_prim_test_between[i32, 32768] 145.7 µs 174.6 µs -16.58%
Simulation old_bp_prim_test_between[i64, 32768] 477.8 µs 538.9 µs -11.34%

Comparing ji/remove-dep (ed9c6f7) with develop (cda251b)

Open in CodSpeed

Footnotes

  1. 1522 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant